-
Notifications
You must be signed in to change notification settings - Fork 17
Enterprise Search snippets #386
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: search/public-preview
Are you sure you want to change the base?
Enterprise Search snippets #386
Conversation
…ebenpae/enterprise-search # Conflicts: # .evergreen-tasks.yml # controllers/operator/mongodbsearch_controller.go # controllers/search_controller/mongodbsearch_reconcile_helper.go # controllers/search_controller/mongodbsearch_reconcile_helper_test.go # controllers/search_controller/search_construction.go # mongodb-community-operator/controllers/replica_set_controller.go
…nto fealebenpae/enterprise-search-snippets
…ebenpae/enterprise-search
…ise-search-snippets
…nto fealebenpae/enterprise-search-snippets
…earch-snippets-refactor
…earch-snippets-refactor
…ise-search-snippets
…earch-snippets-refactor
…ise-search-snippets
…earch-snippets-refactor
There is an existing patch(es) for this commit SHA: Please note that the status that is posted is not in the context of this PR but rather the (latest) existing patch and that may affect some tests that may depend on the particular PR. If your tests do not rely on any PR-specific values (like base or head branch name) then your tests will report the same status. If you would like a patch to run in the context of this PR and abort the other(s), comment 'evergreen retry'. |
0f03679
to
fe32610
Compare
export OPERATOR_ADDITIONAL_HELM_VALUES="registry.imagePullSecrets=prerelease-image-pullsecret" | ||
export OPERATOR_HELM_CHART="oci://quay.io/mongodb/staging/mongodb-kubernetes:${PRERELEASE_VERSION}" | ||
|
||
export OPS_MANAGER_API_URL="${OM_BASE_URL}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we need ops manager env vars in community deployment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no, good catch!
…ise-search-snippets
OPERATOR_ADDITIONAL_HELM_VALUES="$(echo -n "${helm_values[@]}" | tr ' ' ',')" | ||
export OPERATOR_ADDITIONAL_HELM_VALUES | ||
OPERATOR_HELM_CHART="$(realpath "../../../helm_chart")" | ||
OPERATOR_HELM_CHART="$(realpath "${SCRIPT_DIR}/../../helm_chart")" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it work?
For my local testing I had to use
"$(realpath "${SCRIPT_DIR}/../../../helm_chart")"
Based on PR #309
Chain of upstream PRs as of 2025-08-28
PR Search Public Preview - main feature branch #229:
master
←search/public-preview
PR CLOUDP-342325 Basic enterprise search support #309:
search/public-preview
←fealebenpae/enterprise-search
fealebenpae/enterprise-search
←fealebenpae/enterprise-search-snippets
Summary
Proof of Work
Checklist
skip-changelog
label if not needed